feat: add explainSolTransaction using @bitgo/wasm-solana by lcovar · Pull Request #7957 · BitGo/BitGoJS

lcovar · 2026-01-21T22:01:45Z

Summary

Add a standalone explainSolTransaction() function to sdk-coin-sol that uses @bitgo/wasm-solana to explain Solana transactions without depending on @solana/web3.js.

This is a thin ~100-line adapter that replaces the need for the legacy 800+ line Transaction class explain logic. The heavy lifting (parsing, instruction combining, type derivation) happens in @bitgo/wasm-solana — this adapter only handles:

Token name resolution (mint address → sol:usdc via @bitgo/statics)
bigint → string conversion at the serialization boundary
StakingAuthorize → StakingAuthorizeRaw type mapping for downstream compatibility
Mapping to TransactionExplanation interface

Changes

explainTransactionWasm.ts (new): Standalone adapter that calls @bitgo/wasm-solana's explainTransaction(), resolves token names, maps staking authorize fields, and returns TransactionExplanation including inputs and feePayer
transaction.ts: Route tsol through WASM-based explain (validates against production traffic before replacing legacy for all networks)
sol.ts: Add explainTransactionWithWasm() method at coin class level
iface.ts: Add ataOwnerMap and stakingAuthorize to TransactionExplanation interface
instructionParamsFactory.ts: Add Jito stake pool operations (StakePoolDepositSol, StakePoolWithdrawStake)
jitoWasmVerification.ts: New test file verifying Jito liquid staking explain output
transaction.ts tests: Updated assertions for WASM parity (fee defaults to '0' instead of 'UNAVAILABLE', ataOwnerMap, inputs, feePayer, token name via statics)
sol.ts tests: Updated explain assertions to include inputs and feePayer returned by WASM path
webpack config: Add WASM asset handling for @bitgo/wasm-solana
package.json: Add @bitgo/wasm-solana dependency

Key decisions

tsol only: WASM path is gated to testnet (tsol) to validate against production traffic before replacing the legacy path for mainnet
Fee behavior: Returns fee: '0' and feeRate: undefined when fee info not provided (instead of 'UNAVAILABLE'). Downstream already converts UNAVAILABLE → '0', so this is a no-op change
StakingAuthorizeRaw: WASM can decode both raw and non-raw authorize instructions natively, but downstream expects the StakingAuthorizeRaw type name, so the adapter maps it
Token names: Resolved via @bitgo/statics. Tokens not in statics return the raw mint address
inputs/feePayer: Now included in the return so consumers (e.g. wallet-platform) can use the SDK as a single source of truth without calling wasm-solana directly
Test updates: 26 test assertions updated to include inputs and feePayer — these fields were already on the TransactionExplanation interface but weren't populated by the WASM path until now

Depends on

BitGoWASM PR #166 — explainTransaction() with StakingAuthorizeInfo, optional lamportsPerSignature (defaults to 5000n)

Test Plan

All existing sdk-coin-sol tests pass with updated assertions
New Jito WASM verification tests pass
Parity verified against legacy Transaction class output for all transaction types

TICKET: BTC-3025

modules/sdk-coin-sol/src/sol.ts

OttoAllmendinger

we can move towards bigint a little bit more

where external interfaces force us to string keep it

modules/sdk-coin-sol/src/lib/wasm/transaction.ts

modules/sdk-coin-sol/src/lib/wasmInstructionMapper.ts

modules/sdk-coin-sol/src/sol.ts

modules/sdk-coin-sol/src/lib/wasm/transaction.ts

OttoAllmendinger

counting on you claude

modules/sdk-coin-sol/src/sol.ts

Add a standalone `explainSolTransaction()` function that calls @bitgo/wasm-solana's `explainTransaction()` directly and resolves token names via `@bitgo/statics`. This is a thin adapter (~40 lines) that: - Converts wasm-solana's bigint amounts to strings at the boundary - Resolves mint addresses to token names via @bitgo/statics - Maps to the existing TransactionExplanation interface No legacy code is modified. The adapter coexists alongside the existing Transaction class and builders. For tsol (testnet), explainTransaction() routes through the WASM adapter. For sol (mainnet), the legacy builder path is preserved. Changes: - sol.ts: add explainSolTransaction() + explainTransactionWithWasm() - iface.ts: add inputs, feePayer, ataOwnerMap to TransactionExplanation - instructionParamsFactory.ts: export findTokenName (used by adapter) - package.json: add @bitgo/wasm-solana dependency - webpack config: add WASM asset handling - New tests: Jito WASM verification + explain tests for all tx types BTC-3025 TICKET: BTC-0

Follow the *Wasm.ts naming convention used by abstract-utxo (explainPsbtWasm.ts, signPsbtWasm.ts). Moves the standalone WASM explain adapter out of sol.ts into its own module with a dedicated interface, barrel-exported from lib/index.ts. BTC-0 TICKET: BTC-0

Route tsol transactions through the WASM-based explainSolTransaction() in Transaction.explainTransaction(), validating the WASM path against production traffic before replacing the legacy implementation for all networks. Changes: - transaction.ts: tsol uses explainSolTransaction() instead of the legacy switch/case block (no @solana/web3.js dependency) - explainTransactionWasm.ts: add StakingAuthorizeRaw type mapping and stakingAuthorize field support for full parity with legacy - explainTransactionWasm.ts: return fee '0' with feeRate undefined when feeInfo is not provided (matches downstream expectations) - Update test assertions for WASM output shape (ataOwnerMap, fee values, unresolvable token names) BTC-0 TICKET: BTC-0

Populate the inputs and feePayer fields in explainSolTransaction so consumers (wallet-platform) can call the SDK instead of wasm-solana directly. Update 26 test assertions to expect these new fields. BTC-3025 TICKET: BTC-0

lcovar force-pushed the BTC-0-draft-example-sol-decoding branch 27 times, most recently from d32c089 to bb885ad Compare January 28, 2026 18:16

github-advanced-security bot found potential problems Jan 28, 2026

View reviewed changes

modules/sdk-coin-sol/src/sol.ts Fixed Show fixed Hide fixed

lcovar force-pushed the BTC-0-draft-example-sol-decoding branch 2 times, most recently from f5732ce to 97c511d Compare January 28, 2026 18:36

lcovar requested a review from Marzooqa February 10, 2026 19:39

OttoAllmendinger requested changes Feb 11, 2026

View reviewed changes

OttoAllmendinger reviewed Feb 11, 2026

View reviewed changes

modules/sdk-coin-sol/src/lib/wasm/transaction.ts Outdated Show resolved Hide resolved

lcovar force-pushed the BTC-0-draft-example-sol-decoding branch 2 times, most recently from 38e4bc7 to da3dc95 Compare February 11, 2026 19:20

lcovar requested a review from OttoAllmendinger February 11, 2026 19:25

lcovar force-pushed the BTC-0-draft-example-sol-decoding branch 3 times, most recently from 0b3225b to 3cb4054 Compare February 12, 2026 20:22

OttoAllmendinger requested changes Feb 13, 2026

View reviewed changes

modules/sdk-coin-sol/src/sol.ts Outdated Show resolved Hide resolved

modules/sdk-coin-sol/src/sol.ts Outdated Show resolved Hide resolved

modules/sdk-coin-sol/src/sol.ts Outdated Show resolved Hide resolved

modules/sdk-coin-sol/src/sol.ts Outdated Show resolved Hide resolved

lcovar force-pushed the BTC-0-draft-example-sol-decoding branch from 3cb4054 to 009f956 Compare February 13, 2026 10:24

lcovar requested a review from OttoAllmendinger February 13, 2026 16:42

lcovar force-pushed the BTC-0-draft-example-sol-decoding branch 2 times, most recently from c071f3c to e1a4c9d Compare February 14, 2026 06:33

lcovar marked this pull request as draft February 15, 2026 05:32

lcovar force-pushed the BTC-0-draft-example-sol-decoding branch from 01ddba7 to 9400a94 Compare February 15, 2026 05:58

lcovar changed the title ~~feat(sdk-coin-sol): add WASM-based transaction parsing via @bitgo/wasm-solana~~ feat: add explainSolTransaction using @bitgo/wasm-solana Feb 15, 2026

lcovar force-pushed the BTC-0-draft-example-sol-decoding branch 4 times, most recently from a59880d to 9e38c95 Compare February 17, 2026 21:51

lcovar marked this pull request as ready for review February 17, 2026 22:36

OttoAllmendinger previously approved these changes Feb 18, 2026

View reviewed changes

lcovar added 4 commits February 18, 2026 13:47

feat: add inputs and feePayer to WASM explain return

c676e77

Populate the inputs and feePayer fields in explainSolTransaction so consumers (wallet-platform) can call the SDK instead of wasm-solana directly. Update 26 test assertions to expect these new fields. BTC-3025 TICKET: BTC-0

lcovar dismissed OttoAllmendinger’s stale review via c676e77 February 18, 2026 21:47

lcovar force-pushed the BTC-0-draft-example-sol-decoding branch from 9e38c95 to c676e77 Compare February 18, 2026 21:47

lcovar requested a review from OttoAllmendinger February 19, 2026 03:27

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

feat: add explainSolTransaction using @bitgo/wasm-solana#7957

feat: add explainSolTransaction using @bitgo/wasm-solana#7957
lcovar wants to merge 4 commits intomasterfrom
BTC-0-draft-example-sol-decoding

lcovar commented Jan 21, 2026 •

edited

Loading

Uh oh!

Uh oh!

OttoAllmendinger left a comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

OttoAllmendinger left a comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

Comments

Conversation

lcovar commented Jan 21, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Summary

Changes

Key decisions

Depends on

Test Plan

Uh oh!

Uh oh!

OttoAllmendinger left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

OttoAllmendinger left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

Comments

lcovar commented Jan 21, 2026 •

edited

Loading